Skip to content

Enhancements (Integration of Download Script / MCP Server / Docs)#108

Closed
AryanAhadinia wants to merge 5 commits into
mainfrom
AryanAhadinia/MCP
Closed

Enhancements (Integration of Download Script / MCP Server / Docs)#108
AryanAhadinia wants to merge 5 commits into
mainfrom
AryanAhadinia/MCP

Conversation

@AryanAhadinia
Copy link
Copy Markdown
Member

No description provided.

@AryanAhadinia AryanAhadinia requested a review from robmsmt April 29, 2026 07:47
@AryanAhadinia AryanAhadinia self-assigned this Apr 29, 2026
@AryanAhadinia AryanAhadinia added the enhancement New feature or request label Apr 29, 2026
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
52.6% Coverage on New Code (required ≥ 80%)
3.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Copy link
Copy Markdown
Contributor

@robmsmt robmsmt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, reviewed on call

target = os.environ["HF_TARGET"]
print(f"Downloading {model} to: {target}", flush=True)
snapshot_download(repo_id=model, local_dir=target, local_dir_use_symlinks=False, max_workers=1)
print("Download complete.", flush=True)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

chmod

@AryanAhadinia
Copy link
Copy Markdown
Member Author

This is also connected to #34.

@AryanAhadinia
Copy link
Copy Markdown
Member Author

Due to massive complete, we close the PR for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants